Skip to content

Clarify behavioral test scope and stabilize Brooks import test - #299

Merged
jackhai9 merged 2 commits into
mainfrom
codex/document-behavioral-test-policy
Sep 21, 2026
Merged

jackhai9 merged 2 commits into
mainfrom
codex/document-behavioral-test-policy

Conversation

@jackhai9

@jackhai9 jackhai9 commented Sep 21, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • clarify that the repository behavioral principles also apply to services, workers, CLIs, and libraries while keeping userscripts-specific tooling local
  • require an executable Given/When/Then acceptance scenario before production behavior changes
  • document that review owns scenario meaning and test-first ordering while lint enforces only mechanical structure
  • set explicit Brooks import fixture mtimes so the test models browser download order independently of filesystem timestamp coalescing

Validation

  • targeted Brooks import test repeated 20/20 — passed
  • npm run lint:tests — passed
  • npm test — 2087 passed
  • npm run test:affected -- --base d89445d537ffe230ff9ca2c79ccb9e1a68f5d531 — 1174 Node tests and 372 Playwright tests passed
  • git diff --check — passed
  • no userscript source or generated artifact changed; no version bump required

@jackhai9 jackhai9 changed the title Clarify behavioral test scope Clarify behavioral test scope and stabilize Brooks import test Sep 21, 2026
@jackhai9
jackhai9 merged commit 20509d2 into main Sep 21, 2026
1 check passed
@jackhai9
jackhai9 deleted the codex/document-behavioral-test-policy branch September 21, 2026 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant